Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update whois.audns.net.au to match new response format #100

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kovalevsky
Copy link

@kovalevsky kovalevsky commented Aug 7, 2018

AU domains parser is outdated and completely broken now. See #96.
I hope the PR fixes these problems.

Also it moves parser/scanner/specs/etc for whois.audns.net.au to new whois.auda.org.au host according this update weppos/whois@b82bde0#diff-02586b3de517e7049ad4ecd2be67c6deL359

Move files from whois.auda.org.au to whois.auda.org.au, rename classes.
@@ -0,0 +1,2 @@
#status
%s == ["serverRenewProhibited https://afilias.com.au/get-au/whois-status-codes#serverRenewProhibited"]
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suppose that spec is outdated. Or it requires another domain.

@shaneshort
Copy link

is there anything I can do to get this merged? I've been running a seperate fork for just this fix for a few years now

@jarthod
Copy link

jarthod commented Dec 14, 2021

@shaneshort not much here as the maintainer paused work on this project, that's why for the record I ended up starting my fork in which I merged some of the pending PRs, including this one (thanks @kovalevsky!) slightly adapted in 1d7b586, and many more fixes. So people who need this fix but also want other fixes and a more maintained version can use my fork in the meantime:

gem 'whois-parser', github: 'jarthod/whois-parser' # Unofficial but more up-to-date fork, check status at https://github.com/jarthod/whois-parser

Hopefully we'll be able to merge this upsteam someday and improve this situation for everybody (that's why I did not publish another gem yet), If not I'll probably end up publishing another gem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants